Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix breadcrumbs #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix breadcrumbs #7

wants to merge 2 commits into from

Conversation

raprice
Copy link

@raprice raprice commented Dec 12, 2023

This PR brings breadcrumb implementation into alignment with platform architecture requirements. This PR is part of a previously announced batch of breadcrumb fixes across the Learn platform and will be auto-merged if there are no build warnings. This PR may include removing the “extend breadcrumb” feature from any docfx files that are still using it, fixing breadcrumb file references in the docfx file, and rewriting breadcrumb files to match the approved breadcrumb pattern for a given product’s documentation.

Copy link

@raprice : Thanks for your contribution! The author(s) have been notified to review your proposed change. @dotnet-bot

Copy link

Learn Build status updates of commit 831910d:

⚠️ Validation status: warnings

File Status Preview URL Details
dotnet/docfx.json ⚠️Warning Details
dotnet/breadcrumb/toc.yml ✅Succeeded n/a (file deleted or renamed)
dotnet/community-toolkit-dotnet-breadcrumb/toc.yml ✅Succeeded

dotnet/docfx.json

  • Line 47, Column 36: [Warning: ms-prod-technology-invalid - See documentation] Invalid value for 'ms.prod': 'community-toolkit'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@raprice raprice marked this pull request as ready for review December 12, 2023 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant